Skip to content

DevTools standalone shell changes: - #17213

Merged
bvaughn merged 2 commits into
react:masterfrom
bvaughn:devtools-update-react-native-connect-text
Oct 31, 2019
Merged

bvaughn merged 2 commits into
react:masterfrom
bvaughn:devtools-update-react-native-connect-text

Conversation

@bvaughn

@bvaughn bvaughn commented Oct 29, 2019 •

Copy link
Copy Markdown
Contributor

Cosmetic changes to the RN shell:

  1. Update React Native wording to mention recent changes to how DevTools and React Native are meant to connect.
  2. Tweak styles slightly.

Waiting to connect

Before

Screen Shot 2019-10-29 at 1 36 30 PM

After

Screen Shot 2019-10-29 at 1 36 51 PM

Error

Before

Screen Shot 2019-10-29 at 1 42 00 PM

After

Screen Shot 2019-10-29 at 1 57 25 PM

1. Update React Native connection wording
2. Tweak styles slightly

@threepointone threepointone left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarding the second screen; would it be a good idea to provide actionable advice? It’s not apparent it’s an error with that styling. Maybe “Please stop the other running instance” or something?

Feel free to disregard, or do in a future PR. It simply stood out to me in this one.

@bvaughn

bvaughn commented Oct 29, 2019

Copy link
Copy Markdown
Contributor Author

Hm. I guess we could tweak the wording a bit to say something more like:

Another instance of DevTools is running. Only one copy of DevTools can be used at a time.

@sizebot

sizebot commented Oct 29, 2019 •

Copy link
Copy Markdown

Size changes (experimental)

No significant bundle size changes to report.

Generated by 🚫 dangerJS against ef077d4

@sizebot

sizebot commented Oct 29, 2019 •

Copy link
Copy Markdown

Size changes (stable)

No significant bundle size changes to report.

Generated by 🚫 dangerJS against ef077d4

@bvaughn

bvaughn commented Oct 29, 2019

Copy link
Copy Markdown
Contributor Author

Here. This is maybe a little better.
Screen Shot 2019-10-29 at 1 57 25 PM

@bvaughn
bvaughn merged commit 273679a into react:master Oct 31, 2019
@bvaughn
bvaughn deleted the devtools-update-react-native-connect-text branch October 31, 2019 16:22
trueadm pushed a commit to trueadm/react that referenced this pull request Nov 4, 2019
* DevTools standalone shell changes:

1. Update React Native connection wording
2. Tweak styles slightly

* Tweaked standalone DevTools error warning text
mrizwanashiq pushed a commit to mrizwanashiq/react that referenced this pull request Jun 25, 2026
* DevTools standalone shell changes:

1. Update React Native connection wording
2. Tweak styles slightly

* Tweaked standalone DevTools error warning text
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants